-
Notifications
You must be signed in to change notification settings - Fork 131
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[CM-7218] introduce new monkey patching to comet llm #30
Merged
alexkuzmik
merged 20 commits into
main
from
CM-7218-introduce-new-monkey-patching-to-comet-llm
Aug 7, 2023
Merged
[CM-7218] introduce new monkey patching to comet llm #30
alexkuzmik
merged 20 commits into
main
from
CM-7218-introduce-new-monkey-patching-to-comet-llm
Aug 7, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
alexkuzmik
added a commit
that referenced
this pull request
Aug 9, 2023
* Add prototype for import_hooks * Add new entities * Rename module * Make it work on psutil example.Hooray * Fix wrapper * Add copyrights * Fix lint errors * Add docstrings to registry * Update docstrings * Fix lint errors * Add new tests for patching functions * Add tests * Add validate.py and tests for it * Fix lint errors * Add tests * Update tests * Update tests * Add test for child class method, refactor tests a bit * Remove currently odd import * Replace wraps workaround with actual wraps
alexkuzmik
added a commit
that referenced
this pull request
Aug 10, 2023
* Add tests for chain nodes and prompts * Fix some lint errors * Implement chain, move time logic to Timer class, update tests * Add proper support for category * Add state.get_new_id() * input_ and output_ metadata -> metadata, add more type hints * Add some type hints * Fix lint errors * Add tests for Timer * Small refactor * Change error text * Rename input -> inputs in class/function signatures * Init test for node_data_to_dict function * Implement convert.node_data_to_dict * Use node_data_to_dict in ChainNode * Implement node name logic * Add name to node_data_to_dict * Add name to as_dict logic in prompt and node * Fix lint errors * Reorganize some code * Remove node data to dict, move this logic to ChainNode.as_dict() * Rename group_stack -> context * Delete modules that are no longer needed. Rename some of the rest * Fix some lint errors * Add start() to timer * Fix lint errors * Move _timer.start() to __enter__ method * Fix timer usage in chain * Fix lint errors * Add context logic to chain and span * Add span test * Init tests for api, add tests for state. Refactor state.py * Fix lint errors * Add tests for start_chain and end_chain * Implement chains api * Fix lint errors * Move __all__ to comet_llm init * Fix asset format * Fix lint errors * Fix tests after merge conflict * duration -> chain_duration * Add parameters logging to end_chain method * Fix bug in end_chain * Add logic for converting chain inputs and outputs to dicts * Add deepmerge function * Add test for deepmerge * Move Span name to comet_llm namespace * Fix merge conflicts * Rename test * Add summary logic for chains * Fix lint errors * Add category to chain * [CM-7509] llm add deepmerge for input and output metadata (#27) * Add deepmerge logic to chain set_outputs * Fix lint errors * Add tags logging to chains. Log chain_duration as a metric in chains (#28) * Add docstrings to Chain and Span * [CM-7772] llm add a proper asset upload error message for on prem backend users (#34) * Implement check for on-prem in request exceptions handler * Fix lint errors * Refactor * Refactor request exception wrapper * Add files via upload * added gif to readme * Update packaging README with UI Gif * [CM-7218] introduce new monkey patching to comet llm (#30) * Add prototype for import_hooks * Add new entities * Rename module * Make it work on psutil example.Hooray * Fix wrapper * Add copyrights * Fix lint errors * Add docstrings to registry * Update docstrings * Fix lint errors * Add new tests for patching functions * Add tests * Add validate.py and tests for it * Fix lint errors * Add tests * Update tests * Update tests * Add test for child class method, refactor tests a bit * Remove currently odd import * Replace wraps workaround with actual wraps * Update README.md (#32) * [NA] Readme: project_name -> project (#35) * project_name -> project * Fix table * Update Readme.md to add an example logging chain * Cut project_name argument (#37) --------- Co-authored-by: Sid Mehta <[email protected]> Co-authored-by: Boris Feld <[email protected]> Co-authored-by: Elvis Saravia <[email protected]>
andrescrz
deleted the
CM-7218-introduce-new-monkey-patching-to-comet-llm
branch
September 2, 2024 11:35
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.